Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update generic PETG 1.75mm temperatures #251

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

louisrousseau
Copy link

The included 1.75mm temperature values are much lower than those recommended on the different PETG filament boxes I have on hand.

Took values from the 2.85mm generic in generic_petg.xml.fdm_material which were last updated in PR #161 two years ago and also fit within what's on the aforementioned boxes.

@github-actions github-actions bot added the PR: Community Contribution 👑 Community Contribution PR's label May 24, 2023
@louisrousseau
Copy link
Author

@wawanbreton any chance this PR gets merged?

@wawanbreton
Copy link
Contributor

Hi @louisrousseau,
This is quite a big change because it impacts a generic material. I will discuss that with our material experts team.

@wawanbreton
Copy link
Contributor

So we had a conversation about this change, and this is the conclusion:

So if possible, could you make an associated PR on the Cura repository, with the following changes:

  • Identify and fix all the profiles that actually change one of the concerned temperatures
  • Do not change the profiles that use the base temperatures, they will take advantage of this change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: Community Contribution 👑 Community Contribution PR's
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants